Adwaita: color the selection of entries with error...
authorLapo Calamandrei <calamandrei@gmail.com>
Sat, 25 Jun 2016 16:29:51 +0000 (18:29 +0200)
committerLapo Calamandrei <calamandrei@gmail.com>
Sat, 25 Jun 2016 16:29:51 +0000 (18:29 +0200)
...or warning style class applied.
This particular style bit wasn't converted to the saner 3.20 way
so `entry:selected` used in place of `entry selection`.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=768025

gtk/theme/Adwaita/_common.scss
gtk/theme/Adwaita/gtk-contained-dark.css
gtk/theme/Adwaita/gtk-contained.css

index 94f5c45d2dceb3c824c693f2e3c9e3791461df6f..1f08eb38b8790b7ed17987e237df5ccc9b84c09e 100644 (file)
@@ -285,7 +285,7 @@ entry {
 
     &:backdrop:disabled { @include entry(backdrop-insensitive); }
 
-    selection { &:focus, & { @extend %selected_items; }}
+    selection { @extend %selected_items; }
 
     // entry error and warning style
     @each $e_type, $e_color in (error, $error_color),
@@ -296,7 +296,7 @@ entry {
 
         &:focus { @include entry(focus, $e_color); }
 
-        &:selected { &:focus, & { background-color: $e_color; }}
+        selection { background-color: $e_color; }
       }
     }
 
index 9cdf9ba553a5a92baf7fe8cb38b4d87a5a107bbc..2723363e43bb77256dfbce5bce988722720d3a2d 100644 (file)
@@ -238,9 +238,8 @@ entry {
     entry.error:focus {
       box-shadow: inset 0 0 0 1px #cc0000;
       border-color: #1a0000; }
-    spinbutton.error:selected:focus:not(.vertical), spinbutton.error:selected:not(.vertical),
-    entry.error:selected:focus,
-    entry.error:selected {
+    spinbutton.error:not(.vertical) selection,
+    entry.error selection {
       background-color: #cc0000; }
   spinbutton.warning:not(.vertical),
   entry.warning {
@@ -250,9 +249,8 @@ entry {
     entry.warning:focus {
       box-shadow: inset 0 0 0 1px #f57900;
       border-color: #432100; }
-    spinbutton.warning:selected:focus:not(.vertical), spinbutton.warning:selected:not(.vertical),
-    entry.warning:selected:focus,
-    entry.warning:selected {
+    spinbutton.warning:not(.vertical) selection,
+    entry.warning selection {
       background-color: #f57900; }
   spinbutton:not(.vertical) image,
   entry image {
@@ -4518,8 +4516,7 @@ textview text:selected:focus,
 iconview text:selected,
 textview text:selected, .view text selection:focus, iconview text selection:focus, .view text selection, iconview text selection,
 textview text selection:focus,
-textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection:focus, spinbutton:not(.vertical) selection,
-entry selection:focus,
+textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection,
 entry selection, modelbutton.flat:selected, popover.background checkbutton:selected,
 popover.background radiobutton:selected,
 .menuitem.button.flat:selected, treeview.view:selected:focus, treeview.view:selected, row:selected, calendar:selected {
@@ -4532,8 +4529,7 @@ popover.background radiobutton:selected,
   iconview text:selected,
   textview text:selected, .view text selection:focus, iconview text selection:focus, .view text selection, iconview text selection,
   textview text selection:focus,
-  textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection:focus, spinbutton:not(.vertical) selection,
-  entry selection:focus,
+  textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection,
   entry selection, modelbutton.flat:selected, popover.background checkbutton:selected,
   popover.background radiobutton:selected,
   .menuitem.button.flat:selected, treeview.view:selected:focus, treeview.view:selected, row:selected, calendar:selected {
index 44932bf4332f442075db554fa8b14d2f58aec167..de75ded51c92dccc5c4415f0c65bcbcf63f0be26 100644 (file)
@@ -238,9 +238,8 @@ entry {
     entry.error:focus {
       box-shadow: inset 0 0 0 1px #cc0000;
       border-color: #cc0000; }
-    spinbutton.error:selected:focus:not(.vertical), spinbutton.error:selected:not(.vertical),
-    entry.error:selected:focus,
-    entry.error:selected {
+    spinbutton.error:not(.vertical) selection,
+    entry.error selection {
       background-color: #cc0000; }
   spinbutton.warning:not(.vertical),
   entry.warning {
@@ -250,9 +249,8 @@ entry {
     entry.warning:focus {
       box-shadow: inset 0 0 0 1px #f57900;
       border-color: #f57900; }
-    spinbutton.warning:selected:focus:not(.vertical), spinbutton.warning:selected:not(.vertical),
-    entry.warning:selected:focus,
-    entry.warning:selected {
+    spinbutton.warning:not(.vertical) selection,
+    entry.warning selection {
       background-color: #f57900; }
   spinbutton:not(.vertical) image,
   entry image {
@@ -4549,8 +4547,7 @@ textview text:selected:focus,
 iconview text:selected,
 textview text:selected, .view text selection:focus, iconview text selection:focus, .view text selection, iconview text selection,
 textview text selection:focus,
-textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection:focus, spinbutton:not(.vertical) selection,
-entry selection:focus,
+textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection,
 entry selection, modelbutton.flat:selected, popover.background checkbutton:selected,
 popover.background radiobutton:selected,
 .menuitem.button.flat:selected, treeview.view:selected:focus, treeview.view:selected, row:selected, calendar:selected {
@@ -4563,8 +4560,7 @@ popover.background radiobutton:selected,
   iconview text:selected,
   textview text:selected, .view text selection:focus, iconview text selection:focus, .view text selection, iconview text selection,
   textview text selection:focus,
-  textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection:focus, spinbutton:not(.vertical) selection,
-  entry selection:focus,
+  textview text selection, flowbox flowboxchild:selected, spinbutton:not(.vertical) selection,
   entry selection, modelbutton.flat:selected, popover.background checkbutton:selected,
   popover.background radiobutton:selected,
   .menuitem.button.flat:selected, treeview.view:selected:focus, treeview.view:selected, row:selected, calendar:selected {